AFAICT, nightly failures are due to Aqua.jl being incompatible with Julia nightly. This breaks CI on main because it reverts a commit that skipped running the broken tests. That's fine because nightly tests are informative but not required.

You maintain so you decide but having failing CI on master just looks bad. It makes a package look poorly maintained, which is a shame when, as in this case, it isn't.

I weigh things actually working and accurate representation to users very highly while I weigh "good PR"/"looking like everything is working" fairly low. So I'll continue to show folks who view the README info on CI status including nightly. I agree that it would be nice to show a green badge when all required checks pass and un-required checks don't pass but it's not worth the complexity and maintenance burden to achieve that.